update lineage_hunter - refactor#257
update lineage_hunter - refactor#257mateusvahl wants to merge 1 commit intopattern-lab:devfrom mateusvahl:dev
Conversation
|
Hi @Mateuspv it looks like this optimization could be applied to other builder/ files besides list_item_hunter.js, is that correct? |
|
Hi @bmuenzenmeyer, Yes. You can remove the "main code" from inside the closure: |
|
Thanks for the clarification. I'd rather see all the extraneous closures removed at once for consistencies sake. If you feel like tackling that, please do so. Else this might get prioritized a bit further down the pile. |
|
I was just researching this issue, and I agree that we should remove them all. @Mateuspv, if you're up for doing all of them, that would be amazing, but if you don't want to, I could do it as part of a larger whitespace & indentation cleanup. |
|
@geoffp I'm sure when I will do it =/, so feel free to do.. |
|
I second rolling it into #264 |
|
Added as a todo item to #264. Closing this. |
Target the
devbranch!Summary of changes:
I remove Closures("(function () { ... })()") since node will not export any that are not implicit with
module.exports. Also removed some nested functions.Thanks!